Skip to content

Add a user manual#18

Open
colincoleman wants to merge 3 commits into
xyzzy42:masterfrom
colincoleman:feat-user-manual
Open

Add a user manual#18
colincoleman wants to merge 3 commits into
xyzzy42:masterfrom
colincoleman:feat-user-manual

Conversation

@colincoleman

Copy link
Copy Markdown

This update adds an AI-generated user manual (docs/user-manual.md). The content was derived from the project README, source code constants, and general knowledge of mechanical watch timing from the AI's training data.

The README is updated to link to the new manual.

This update adds an AI-generated user manual (docs/user-manual.md). The
content was derived from the project README, source code constants, and
general knowledge of mechanical watch timing from the AI's training data.

The README is updated to link to the new manual.
@xyzzy42

xyzzy42 commented Jun 20, 2026

Copy link
Copy Markdown
Owner

Having the content of the README in two places does not seem like an improvement to me. It's just my words, AI sloppified and with errors added, being repeated again. Is it helpful to write the same thing in two places, mostly the same, but also different? What happens when one is changed, but not the other?

Comment thread docs/user-manual.md Outdated

## 1. Introduction

tg-timer is an open-source program for timing mechanical watches. It listens to the

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

open source does not have a hyphen.

@xyzzy42 xyzzy42 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just too AI sloppy to be of use or something I want to deal with as it is now.

Comment thread docs/user-manual.md Outdated
## 2. Installation

This section provides platform-specific instructions for installing tg-timer. Each
subsection lists the required dependencies, exact package manager commands, build

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Each subsection does NOT list required dependencies. But this should not go here. This is already in the README with fewer errors.

Comment thread docs/user-manual.md Outdated
| 28800 | Common modern frequency (8 beats/sec) |
| 36000 | High-beat movements (10 beats/sec) |
| 43200 | Ultra-high-beat movements |
| 72000 | Spring Drive and similar |

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you have a reference for spring drive being detected on a time grapher at 7200 bph?

Comment thread docs/user-manual.md Outdated
analogous to the paper-strip output of a traditional mechanical timegrapher.

**Timing dots.** Each dot represents one detected beat (a tic or toc event). The
horizontal position of a dot indicates its timing deviation from the ideal interval.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The paperstrip can be either vertical or horizontal.

Comment thread docs/user-manual.md Outdated

| Control | Action |
|---------|--------|
| Zoom in / out | Adjust the horizontal scale to show more or less timing detail per dot |

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"More or less detail per dot"? That makes no sense.

Comment thread docs/user-manual.md Outdated
| Clear | Erase all accumulated timing dots and start fresh |

Zoom is useful when the dots are tightly clustered (zoom in to see detail) or spread
beyond the visible area (zoom out to see the overall pattern). Shift and center help

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The display wraps around. It's not possible to go beyond the visible area.

Comment thread docs/user-manual.md Outdated

### Save Behavior

tg-timer saves settings in two ways:

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again, details that don't belong in a manual.

Comment thread docs/user-manual.md Outdated
In normal mode, tg-timer processes every audio sample at the full sample rate (e.g.,
all 44100 samples per second at the default rate). In Light Algorithm mode, the
application applies decimation by discarding every other sample, effectively cutting
the sample rate in half (e.g., from 44100 Hz down to an effective 22050 Hz). This

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How many times should light mode be described in the manual? I think at most once would be enough.

Comment thread docs/user-manual.md Outdated

On Windows, the Signal and Filter Chain dialogs may fail to open or display
incorrectly. This is a known issue caused by a Python library loading conflict in the
MSYS2 environment — the dialogs depend on Python (for plotting), and certain library

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds like a made up reason.

Comment thread docs/user-manual.md Outdated
the BPH, then confirm the result against the movement's specification.

2. **Perform the calibration procedure.** An uncalibrated audio device can introduce
a systematic drift that appears as wandering Rate values. Follow the calibration

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, calibration does nothing to deal with instability of the audio clock.

Comment thread docs/user-manual.md Outdated
- The exact error message or unexpected behavior observed
- Steps to reproduce the issue

The community and maintainers can provide additional guidance or confirm whether the

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Community? Maintainers, plural?

@colincoleman

colincoleman commented Jun 22, 2026

Copy link
Copy Markdown
Author

I appreciate the desire to keep AI slop at bay. And appreciate that you have spent the time actually reviewing this.
This project has had a README that has stated there is no manual for over 10 years - I would guess the reason for this is that writing a user manual is hard / unrewarding work that no one want to do. It is for this reason I thought it would be nice to start with something.
I would be happy to go through a few editorial rounds and cut / change anything you suggest.

On second thoughts lets start from a simpler place - I have taken your comments into account and have generated a shorter more accurate / less sloppy version

This update proposes a leaner more accurate user manual that does not unncessarily
duplicate info in the README.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants